tools/toolbox: Consistently handle option parsing failures

`getopt` already prints a message on parse errors, but it still
seems like good style to point to --help as we do elsewhere.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3113>

diff --git a/tools/toolbox/create-toolbox.sh b/tools/toolbox/create-toolbox.sh
index ba240c537ceba758c81ce4bf6d8d78c8038c722d..5c06021e01c8f250692dacdb0c76b5fc10c54114 100755 (executable)
--- a/tools/toolbox/create-toolbox.sh
+++ b/tools/toolbox/create-toolbox.sh
@@ -96,7 +96,7 @@ TEMP=$(getopt \
   --longoptions 'skip-mutter' \
   --longoptions 'set-default' \
   --longoptions 'help' \
-  -- "$@")
+  -- "$@") || die "Run $(basename $0) --help to see available options"
 
 eval set -- "$TEMP"
 unset TEMP

diff --git a/tools/toolbox/meson-build.sh b/tools/toolbox/meson-build.sh
index 0f5923bb8a5f5e37a881dbd579526a20a36817ea..82234f1197c1cc61ffc8e5b0f061d041946bcadb 100755 (executable)
--- a/tools/toolbox/meson-build.sh
+++ b/tools/toolbox/meson-build.sh
@@ -50,7 +50,7 @@ TEMP=$(getopt \
   --longoptions 'toolbox:' \
   --longoptions 'dist' \
   --longoptions 'help' \
-  -- "$@")
+  -- "$@") || die "Run $(basename $0) --help to see available options"
 
 eval set -- "$TEMP"
 unset TEMP